Skip to content

docs(self-hosted): introduce troubleshooting snuba page#17782

Merged
aldy505 merged 9 commits into
masterfrom
aldy505/self-hosted/troubleshooting-snuba
Jun 24, 2026
Merged

docs(self-hosted): introduce troubleshooting snuba page#17782
aldy505 merged 9 commits into
masterfrom
aldy505/self-hosted/troubleshooting-snuba

Conversation

@aldy505

@aldy505 aldy505 commented May 18, 2026

Copy link
Copy Markdown
Collaborator

Closes getsentry/self-hosted#4157

Most self-hosted users most likely won't read Snuba's documentation (https://getsentry.github.io/snuba/) and it mostly only includes technical stuff related to ClickHouse, but not the thoughts behind them.

@vercel

vercel Bot commented May 18, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
develop-docs Ready Ready Preview, Comment Jun 23, 2026 11:07pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
sentry-docs Ignored Ignored Preview Jun 23, 2026 11:07pm

Request Review

Comment thread develop-docs/self-hosted/troubleshooting/snuba.mdx
sfanahata added a commit that referenced this pull request Jun 23, 2026
## Summary

The `crate-ci/typos` checker splits `INSERTs` and `SELECTs` into stems
`INSER` and `SELEC`, then flags them as misspellings of `INSERT` and
`SELECT`. These are valid pluralized SQL keywords used in technical
documentation (e.g., "ClickHouse INSERTs").

This adds both words to the `[default.extend-words]` allowlist in
`_typos.toml`, following the same pattern already used for `DSNs`,
`SDKs`, `APIs`, etc.

## Motivation

Unblocks #17782, which is blocked by the required `Check Typos` CI check
due to these false positives on line 89 of
`develop-docs/self-hosted/troubleshooting/snuba.mdx`.

Co-authored-by: Shannon Anahata <shannonanahata@gmail.com>
sfanahata added a commit that referenced this pull request Jun 23, 2026
## Summary

Follow-up to #18520. The full-word entries `INSERTs`/`SELECTs` added in
that PR are not sufficient because `crate-ci/typos` splits mixed-case
plurals at the case boundary (`INSER` + `Ts`) and checks each subword
independently. The stems `INSER` and `SELEC` must also be allowlisted.

This follows the same pattern already used in the config:
- `STARTD` — stem of `MIGRATED_GETTING_STARTD_DOCS`
- `ERRO` — stem of `ERRORs`

## Motivation

Unblocks #17782, which is still blocked by the `Check Typos` CI check
after #18520 was merged.

Co-authored-by: Shannon Anahata <shannonanahata@gmail.com>
Comment thread develop-docs/self-hosted/troubleshooting/snuba.mdx Outdated
- Change sidebar_order from 2 to 6 to avoid conflict with kafka.mdx
  (existing pages use orders 1-5)
- Fix subject-verb agreement: 'ingest' -> 'ingests'
- Fix spelling: 'queryer' -> 'querier'
- Fix phrasing: 'returns the results into' -> 'returns the results to'
- Remove double space before backtick-quoted text
Comment thread develop-docs/self-hosted/troubleshooting/snuba.mdx Outdated
The Kafka CLI commands (kafka-console-consumer, kafka-consumer-groups)
need to run inside the Kafka container in the self-hosted Docker Compose
environment. Add 'docker compose exec kafka' prefix and
'--bootstrap-server kafka:9092' flag to match the pattern used in
kafka.mdx. Also normalize 'kafka-consumer-groups.sh' to
'kafka-consumer-groups' (no .sh suffix) for consistency.
Comment thread develop-docs/self-hosted/troubleshooting/snuba.mdx Outdated
The kafka-consumer-groups --describe command requires either --group or
--all-groups to be specified. Add --all-groups since this is a general
troubleshooting hint.
Comment thread develop-docs/self-hosted/troubleshooting/snuba.mdx Outdated
The actual consumer group used by snuba-subscription-consumer-events in
docker-compose.yml is 'snuba-events-subscriptions-consumers', not
'snuba-events-subscription-executor'.

@sfanahata sfanahata left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤘

@aldy505 aldy505 merged commit 46b12f2 into master Jun 24, 2026
21 checks passed
@aldy505 aldy505 deleted the aldy505/self-hosted/troubleshooting-snuba branch June 24, 2026 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clarify behavior of Snuba subscription consumers

2 participants